Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed hang on exit of isolate select with geom subsets, with test. #234

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

ppt-adsk
Copy link
Collaborator

@ppt-adsk ppt-adsk commented Dec 17, 2024

When a point instance prototype has a geom subset under a mesh, exiting isolate select causes a hang in Hydra Storm. Fixed by avoiding setting visibility off on geom subsets, which is irrelevant anyways for isolate select.

@ppt-adsk ppt-adsk self-assigned this Dec 17, 2024
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without the fix, this test hangs. With the fix, it passes.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All references to Maya 2026 removed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test data taken from HYDRA-1242 ticket, belongs to Autodesk.

@ppt-adsk ppt-adsk assigned ppt-adsk and unassigned ppt-adsk Jan 16, 2025
@ppt-adsk ppt-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed do-not-merge-yet labels Jan 16, 2025
@lilike-adsk lilike-adsk merged commit eb243e9 into dev Jan 17, 2025
10 checks passed
@lilike-adsk lilike-adsk deleted the tremblp/HYDRA-1242/isolate_select_hang branch January 17, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants